• Please bump the PKG_RELEASE by one.
• Please add a valid Signed-off-by with your correct name.
• Please add valid commit messages why we need to add ':' to the CHARSET so we could see in Git why we have to add this. A reference to the issue is not enough

Please add a space between your last name and first name, thanks.

Please add a space between your last name and first name, thanks.

What if there is only first name?

@feckert @1715173329 What else do I need to do?

I do not speak Chinese! But deepl.com says '行旅途' means 'travel destination'.
And I don't know how the names are constructed in china.
CI/CD still complains
These are the rules sorry

CI/CD still complains
These are the rules sorry

How do I change to fit the rules?

Sign DCO with your legal name, at least, looks legal.
And nowadays I don't think any legal Chinese name does not contain last name.
I guess "行旅途" means "go travel".

Sign DCO with your legal name, at least, looks legal.

Can I only sign with my legal name? I would prefer to contribute anonymously.

These are the rules sorry

These are the rules sorry

sad

@feckert @1715173329 If you have time please approve the inspection.

Looks good. Just a little nit pick, then we can merge. Can you please limit the commit message to 80 characters per line and remove the second mention of #25051. This does not need to be written twice in the commit message.

...and remove the second mention of #25051. This does not need to be written twice in the commit message.

If you look in git, it's actually a github issue link and the string #25051. I'll keep only one if you ask me to keep only one though.

Using git cmd on the shell I see:

IPv6 is separated by `:` instead of `.`, so we need to add `:` in DNS_CHARSET to fix issue #25051
or https://github.com/openwrt/packages/issues/25051

On the github WebUI I see:

IPv6 is separated by `:` instead of `.`, so we need to add `:` in DNS_CHARSET to fix issue #25051
or #25051

Please write https://github.com/openwrt/packages/issues/25051 so no github user also can see the link. On the github WebUI the link is auto resolved. Therefore #25051 appears twice.

Also I see in my git history:
2024-09-30 03:46 +0800 行旅途 o [pr/upstream-25056] ddns-scripts: add colon char in DNS_CHARSET

But in the commit message you have added the line
Signed-off-by: Xiaolong Zhang <xliilQwQ@outlook.com>

I think if I start running the CI/CD then this will not be happy and will complain Test Formalities / Test Formalities (pull_request)
You have to update your .gitconfig with the name in your Signed-off-by line so we have.

[user]
name = Xiaolong Zhang
email = xliilQwQ@outlook.com

Then add the change wit git add <file> and after that execute git commit -s. This will add a valid Signed-off-by and the correct auther to the repository so we have.

2024-09-30 03:46 +0800 Xiaolong Zhang o [pr/upstream-25056] ddns-scripts: add colon char in DNS_CHARSET